Remove call outs per style guide#24238
Conversation
✅ Deploy Preview for docsdocker ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
github-actions
bot
added
the
area/subscription
|
/review |
aevesdocker
left a comment
aevesdocker
left a comment
There was a problem hiding this comment.
Great cleanup overall, just a few suggestions
.claude/settings.local.json
Outdated
| > [!TIP] | ||
| > | ||
| > If users have issues starting Docker Desktop after enforcing sign-in, | ||
| If users have issues starting Docker Desktop after enforcing sign-in, |
There was a problem hiding this comment.
I actually quite like this as a tip. There is an 'Important' CTA further down that talks about desktop versions which can instead be removed as we nolonger support those mentioned versions 🤝
| {{< tabs >}} | ||
| {{< tab name="Docker subscription" >}} | ||
|
|
||
| > [!IMPORTANT] |
There was a problem hiding this comment.
I understand why you've made this change, but I think it actually gets lost moving it out of the callout 🤔
Maybe it's worth having a combined callout at the top saying "If you have a sales-assisted Docker Business subscription, contact your account manager to add or remove seats to your subscription".
I'd then remove the tax callout (pretty sure you'll get the go-ahead to remove it anyway)
There was a problem hiding this comment.
ok, sweet. i'll make a follow up task to figure out what to do with all of these tax compliance call outs tbh.
3 participants
This PR captures work for docs in BRIDGE that have >2 but <5 CTAs per doc.
23.Follow-up work: